Skip to content

chore: update#1

Open
dennisme wants to merge 1 commit intomasterfrom
dennisme/update
Open

chore: update#1
dennisme wants to merge 1 commit intomasterfrom
dennisme/update

Conversation

@dennisme
Copy link
Copy Markdown

@dennisme dennisme commented Jan 14, 2024

Makes a number of updates:

  • newer golang
  • updates all go packages
  • moves from syscall to golang.org/x/sys/unix
  • makes BUILDDATE work on both unix and mac systems
  • makes uses of exporter-toolkit's standard flags (web.listen-address and web.config.file)
  • fixes some broken links and removes trailing spaces from a handful of lines
  • ran go clean -modcache and go mod tidy

@dennisme dennisme requested a review from wbollock January 14, 2024 06:37

## Exposing metrics through HTTPS

Process exporter supports TLS and basic auth through using the --web.config.file format described below and in the [exporter-toolkit repository](https://github.com/prometheus/exporter-toolkit/blob/master/docs/web-configuration.md).
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Process exporter supports TLS and basic auth through using the --web.config.file format described below and in the [exporter-toolkit repository](https://github.com/prometheus/exporter-toolkit/blob/master/docs/web-configuration.md).
Process exporter supports TLS and basic auth through using the --web.config.file format described below and in the [exporter-toolkit repository](https://github.com/prometheus/exporter-toolkit/blob/3f0df9ed8ef420d6000d64638f5fe6c4e7b71912/docs/web-configuration.md).

permalink

```

For further information about TLS configuration, please visit: [exporter-toolkit](https://github.com/prometheus/exporter-toolkit/blob/master/docs/web-configuration.md) No newline at end of file
For further information about TLS configuration, please visit: [exporter-toolkit](https://github.com/prometheus/exporter-toolkit/blob/master/docs/web-configuration.md)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
For further information about TLS configuration, please visit: [exporter-toolkit](https://github.com/prometheus/exporter-toolkit/blob/master/docs/web-configuration.md)
For further information about TLS configuration, please visit: [exporter-toolkit](https://github.com/prometheus/exporter-toolkit/blob/3f0df9ed8ef420d6000d64638f5fe6c4e7b71912/docs/web-configuration.md)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants